Skip to content
This repository was archived by the owner on Jun 25, 2026. It is now read-only.

Make use of the new name filtering in calls to list jobs.#595

Merged
lennartkats-db merged 1 commit into
databricks:mainfrom
dinowernli:add-name-filter-to-jobs-list
Jan 3, 2023
Merged

Make use of the new name filtering in calls to list jobs.#595
lennartkats-db merged 1 commit into
databricks:mainfrom
dinowernli:add-name-filter-to-jobs-list

Conversation

@dinowernli

@dinowernli dinowernli commented Dec 27, 2022

Copy link
Copy Markdown
Contributor

Tried this out manually:

> databricks jobs list --name=test_continuous --version=2.1
371294009417906  test_continuous

And with 2.0 instead of 2.1:

> databricks jobs list --name=test_continuous
WARN: Your CLI is configured to use Jobs API 2.0. In order to use the latest Jobs features please upgrade to 2.1: 'databricks jobs configure --version=2.1'. Future versions of this CLI will default to the new Jobs API. Learn more at https://docs.databricks.com/dev-tools/cli/jobs-cli.html
ERROR: the options --expand-tasks, --offset, --limit, --all, and --name are only available in API 2.1

@dinowernli dinowernli force-pushed the add-name-filter-to-jobs-list branch from 0900233 to 11deb07 Compare December 29, 2022 14:29
@codecov-commenter

codecov-commenter commented Dec 29, 2022

Copy link
Copy Markdown

Codecov Report

Base: 75.80% // Head: 75.81% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (202ceca) compared to base (fac68b7).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #595   +/-   ##
=======================================
  Coverage   75.80%   75.81%           
=======================================
  Files          55       55           
  Lines        4922     4924    +2     
=======================================
+ Hits         3731     3733    +2     
  Misses       1191     1191           
Impacted Files Coverage Δ
databricks_cli/jobs/api.py 95.65% <100.00%> (ø)
databricks_cli/jobs/cli.py 96.12% <100.00%> (+0.05%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dinowernli dinowernli force-pushed the add-name-filter-to-jobs-list branch 2 times, most recently from 4d5671c to af5fec7 Compare December 29, 2022 14:35
@dinowernli dinowernli marked this pull request as ready for review December 29, 2022 14:36

@lennartkats-db lennartkats-db left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please note that you need to enable commit signing for this repository.

@dinowernli dinowernli force-pushed the add-name-filter-to-jobs-list branch from af5fec7 to b673f82 Compare January 3, 2023 12:50
@dinowernli dinowernli force-pushed the add-name-filter-to-jobs-list branch from b673f82 to 202ceca Compare January 3, 2023 12:56
@lennartkats-db lennartkats-db merged commit fd2ad70 into databricks:main Jan 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants